-
Notifications
You must be signed in to change notification settings - Fork 586
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documented and formatted a lot of files to sync with pokeemerald #636
base: master
Are you sure you want to change the base?
Conversation
…ttle_ai_scripts.s
…, wirelessWinTop, wirelessWinRight and wirelessWinBottom respectively
…nd STRINGID_GOTCHAPKMNCAUGHT2 to STRINGID_GOTCHAPKMNCAUGHTOLDMANORDUDE
And pushed more changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Keep in mind when syncing things between pokefirered and pokeemerald that just copying everything over isn't sufficient. There are many names pokefirered uses that are better than the equivalent in pokeemerald, and there are game differences to account for.
…eld_effect_object_tmplate_pointers.h to extern const struct instead of const struct
…tatic bool8 and sMovementActionFuncs to gMovementActionFuncs matching emerald
…h and also changed sItemEffect to gItemEffect
…nst u32 in src/data/pokemon//tmhm_learnsets.h
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#define tWin0Left data[3] | ||
#define tWin0Right data[4] | ||
#define tWin0Top data[5] | ||
#define tWin0Bottom data[6] | ||
#define tWinLeft data[3] | ||
#define tWinRight data[4] | ||
#define tWinTop data[5] | ||
#define tWinBottom data[6] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason for this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason to keep 0 in between these names??
Like there is no tWin1Left, tWin2Left or something
And of course it's consistent with pokeemerald
…to BattleScript_ActionLeftoverWallyPrepToThrow
No description provided.